Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

723 lab 728 lab 721 web socket security error in production #724

Conversation

thetechnocrat-dev
Copy link
Contributor

What type of PR is this? (check all applicable)

  • 🐛 Bug Fix

Description

This error is getting caused by using ws instead of wss in an https enviroment.

Screenshot 2023-10-23 at 4 00 34 PM

This PR fixes it by adding wss when the url is https. Also, added the 5mb req to the equibind gateway tool.

Related Tickets & Documents

Linear Ticket

Steps to Test

Uploading the new equibind gateway file and equibind test files should run against any docker compose full stack env. Logs don't stream until Bacalhau Job starts, so just keep re-clicking the update button if a web socket closed error message appears.

Screenshot 2023-10-23 at 3 57 43 PM

Not sure how to test the wss portion without deploying.

Relevant GIF

Screenshot 2023-10-23 at 4 09 18 PM

@thetechnocrat-dev thetechnocrat-dev linked an issue Oct 23, 2023 that may be closed by this pull request
@linear
Copy link

linear bot commented Oct 23, 2023

LAB-728 [LAB-721] web socket security error in production

Seems like production needs to use wss:// instead of ws://

From SyncLinear.com | LAB-721

LAB-721 web socket security error in production

Seems like production needs to use wss:// instead of ws://

@vercel
Copy link

vercel bot commented Oct 23, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated (UTC)
docs ⬜️ Ignored (Inspect) Oct 23, 2023 8:11pm

@thetechnocrat-dev thetechnocrat-dev temporarily deployed to ci October 23, 2023 20:11 — with GitHub Actions Inactive
@thetechnocrat-dev thetechnocrat-dev temporarily deployed to ci October 23, 2023 20:11 — with GitHub Actions Inactive
@thetechnocrat-dev thetechnocrat-dev temporarily deployed to ci October 23, 2023 20:11 — with GitHub Actions Inactive
@thetechnocrat-dev thetechnocrat-dev temporarily deployed to ci October 23, 2023 20:15 — with GitHub Actions Inactive
@thetechnocrat-dev thetechnocrat-dev temporarily deployed to ci October 23, 2023 20:15 — with GitHub Actions Inactive
Copy link
Collaborator

@alabdao alabdao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thetechnocrat-dev thetechnocrat-dev merged commit d11503c into main Oct 23, 2023
11 checks passed
@thetechnocrat-dev thetechnocrat-dev deleted the 723-lab-728-lab-721-web-socket-security-error-in-production branch October 23, 2023 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[LAB-728] [LAB-721] web socket security error in production
2 participants